home *** CD-ROM | disk | FTP | other *** search
/ Atari Mega Archive 1 / Atari Mega Archive - Volume 1.iso / lists / mint / l_1599 / 1256 < prev    next >
Encoding:
Internet Message Format  |  1994-08-27  |  1.7 KB
  1. From: Kay Roemer <roemer@informatik.uni-frankfurt.de>
  2. Posted-Date: Tue, 29 Mar 94 9:06:20 MESZ
  3. Received-Date: Tue, 29 Mar 94 09:06:20 +0200
  4. Message-Id: <9403290706.AA29864@hera.rbi.informatik.uni-frankfurt.de>
  5. Subject: Re: Mint 1.10: Bug in Fselect()
  6. To: nox@jelal.north.de (Juergen Lock)
  7. Date: Tue, 29 Mar 94 9:06:20 MESZ
  8. In-Reply-To: <9403212113.AA00065@jelal.north.de>; from "Juergen Lock" at Mar 21, 94 10:13 pm
  9. Mailer: Elm [revision: 70.85]
  10.  
  11. Juergen Lock writes:
  12.  
  13. [...]
  14.  
  15. >  ahaa so my last patch should better be changed like this?
  16. > 
  17. > Index: dosfile.c
  18. > @@ -1045,6 +1045,7 @@
  19. >      }
  20. >  
  21. >      if (count == 0) {
  22. > +        ulong onsigs;
  23. >          /* OK, now let's set a timeout */
  24. >  
  25. >          if (timeout) {
  26. > @@ -1057,6 +1058,7 @@
  27. >          sr = spl7();
  28. >  
  29. >      /* curproc->wait_cond changes when data arrives or the timeout happens */
  30. > +        onsigs = curproc->nsigs;
  31. >          while (curproc->wait_cond == (long)wakeselect) {
  32. >  #if 0
  33. >  /* better not call BIOS with interrupts off, especially not Bconin... :) */
  34. > @@ -1068,6 +1070,8 @@
  35. >                  check_sigs();
  36. >                  sr = spl7();
  37. >              }
  38. > +            if (onsigs != curproc->nsigs)
  39. > +                break;
  40. >          }
  41. >          spl(sr);
  42. >  
  43. >  or _must_ check_sigs run at level 7 here?  then stopping the modem1
  44. > receiver overruns could get a bit harder...
  45.  
  46. The only reason I can think of for the spl7()'s beeing there is to avoid
  47. race conditions with:
  48.  
  49.     while (curproc->wait_cond == (long)wakeselect) {
  50.         /* XXX */
  51.         sleep ();
  52.     }
  53.  
  54. because without them an interrupt handler could interrupt at /*XXX*/
  55. and clear curproc->wait_cond, thus making the process sleep forever,
  56. because it is woken up before actually going to sleep.
  57.  
  58. So running check_sigs() not at spl7 should be OK.
  59.  
  60. Cheers, Kay.
  61.